Skip to content

Migrate from poetry to uv #355

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Jun 23, 2025
Merged

Migrate from poetry to uv #355

merged 9 commits into from
Jun 23, 2025

Conversation

tylerhutcherson
Copy link
Collaborator

Beyond poetry dependency resolution being the bane of my existence... uv is just better. Faster, stronger..

This PR introduces uv as our new package manager and greatly improves the economics of our testing CI workflow on a few fronts:

  • Updated versions of common actions
  • DRY(er) environment var setup
  • Isolated dependency installs during CI tests
  • Caching enabled for all of the things

@tylerhutcherson tylerhutcherson force-pushed the feat/RAAE-920-poetry-to-uv branch from 4e7471e to c19c0e1 Compare June 19, 2025 16:40
@tylerhutcherson tylerhutcherson changed the title Migrate from poetry to uv. Migrate from poetry to uv Jun 19, 2025
@tylerhutcherson tylerhutcherson added the enhancement New feature or request label Jun 19, 2025
@tylerhutcherson tylerhutcherson requested a review from rbs333 June 19, 2025 21:49
Copy link
Collaborator

@rbs333 rbs333 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like it

  • Pulled local uv sync worked for me
  • Make additions worked
  • Pre-commit also working for local commit
  • Generally faster seems like an improvement

CONTRIBUTING.md Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good improvements here

Makefile Outdated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same some nice adds

- name: Install Poetry
uses: snok/install-poetry@v1
- name: Install uv
uses: astral-sh/setup-uv@v6
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't have to do this right now, but Astral publishes Docker images with uv already installed in the image, and they're Python version specific, so we could use those as a base image for every test we've tested in the matrix, assuming they have a base image for that version. And that would probably cut down a lot of install time.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will take a look and just give it a try :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ran into a few issues. Going to create a ticket.

@tylerhutcherson tylerhutcherson merged commit 7c7d4f2 into main Jun 23, 2025
33 checks passed
@tylerhutcherson tylerhutcherson deleted the feat/RAAE-920-poetry-to-uv branch June 23, 2025 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants